hack/ci: pass --remote to bud tests - #29529
Merged
Luap99 merged 2 commits intoAug 17, 2026
Merged
Conversation
Luap99
reviewed
Aug 17, 2026
Luap99
left a comment
Luap99
left a comment
Member
There was a problem hiding this comment.
I think passing --remote makes sense but since we no longer set PODBIN_NAME I would usggest you drop that from the script and just use REMOTE/PODMAN_BINARY env there.
Signed-off-by: Tushar Verma <tusharmyself06@gmail.com>
Signed-off-by: Tushar Verma <tusharmyself06@gmail.com>
vtushar06
force-pushed
the
fix-bud-remote-mode
branch
from
August 17, 2026 11:00
e2ca3d0 to
de8c356
Compare
Contributor
Author
|
okay dropped it. @Luap99 now only PODBIN_NAME left is the cirrus yaml text in hack/buildah-vendor-treadmill, left that alone |
Luap99
enabled auto-merge
August 17, 2026 12:35
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
bud remotejob runs the same local binary asbud local. Both jobs from run 31847250452 on main print the identical bats invocation:bin/podman, notpodman-remote, andREMOTE=empty, in the job named remote.The remote path in
run-buildah-bud-testsis gated onPODBIN_NAME(line 78), which Cirrus used to set per matrix task. The migration replaced that with amode:matrix key, butrun_bud()never reads$MODE-run_int,run_sys,run_machineandrun_upgradeall do.grep -rn PODBIN_NAMEoutside vendor finds it only insiderun-buildah-bud-testsitself and in the treadmill script's Cirrus text.So
apply-podman-deltashas 30skip_if_remoteand 4skip_if_rootless_remoteentries that are currently no-ops, and podman-remote build coverage has been off since the migration.--remoteis already an option on the script and setsPODBIN_NAME=remotebefore that gate, so this is one branch inrun_bud.Worth saying plainly: this restores coverage rather than being a no-op, so the remote leg may go red on things that have been hidden. Most of the historically broken ones should already be covered by those deltas, but if new failures show up I am happy to iterate on them here or in a follow-up.